3.22.16

Update a11y tests

Expected output of a11y tests has changed.

Updated Czech translation

Updated Spanish translation

gdk: Set vid/pid on wayland tablets

This was missed so far... Use %.4x format, in order to behave just
the same than X11.

headerbar: fix "widow" typo in docs

Updated Norwegian bokmål translation

GtkActionHelper: Remove unnecessary NULL check

This NULL check has existed since commit 652f16dd985dbc1b, when
GtkActionHelper was first introduced, but so has the assertion at the
top to ensure the validity of 'widget'.

https://bugzilla.gnome.org/show_bug.cgi?id=783587

GtkActionHelper: Allow a NULL action-name to unset the previous GAction

https://bugzilla.gnome.org/show_bug.cgi?id=783587

file chooser: Don't pass NULL as title to the portal

This causes critical warnings.

printunixdialog: Update collate icon as entry changes

By relying on GtkSpinButton default activation behavior, the
collate icon doesn't get updated when a new number is typed
in the copies spin button.

https://bugzilla.gnome.org/show_bug.cgi?id=759308

gtktextdisplay: remove unused code

In render_para() function, which is called for every text line that
needs to be drawn.

https://bugzilla.gnome.org/show_bug.cgi?id=783397

dist icon.list

Otherwise things will not work in a release tarball.

icon-browser: Add a few icons

These have appeared recently.

icon browser: Make icons a bit larger

Use 48x48 in the list, instead of 32x32. We have the space.

icon browser: Add a 'copy to clipboard' button

This makes it more obvious how to use the icon.

icon browser: put icons as data

Instead of hardcoding all the add_icon calls,
read the icon list from a keyfile, for easier
extensibility.

testsuite/scrolledwindow: Try non-overlay/non-auto

It was only testing the default configuration, where overlay scrolling
is on and both scrollbars use POLICY_AUTOMATIC. We should also test the
other 3 configurations that are available by including non-overlay
scrollbars and/or those that use POLICY_ALWAYS.

https://bugzilla.gnome.org/show_bug.cgi?id=778853

ScrolledWindow: Don’t req size for autohidden bars

POLICY_AUTOMATIC means scrollbars are only shown when needed, i.e. when
the size of the window is not large enough to show the entire child. So
when measuring the preferred size, such scrollbars should be ignored.

But measure() added size for *any* non-overlay scrollbar of the opposite
orientation, e.g. for horizontal size, it added the width of vscrollbar.
So we requested for child + bar, & having enough for child meant that the
policy hid the bar, leaving extra space empty below/right of the child.

Fix this by only adding size for such bars if they use POLICY_ALWAYS.

https://bugzilla.gnome.org/show_bug.cgi?id=778853

scrolledwindow: Avoid critical when removing source

Make sure indicator->conceil_timer is actually set before trying to
remove it.

wayland: selectively cancel key repeat on key release

Under Wayland, when multiple keys are pressed and the user releases a
key, key repeat should continue unless the key released is the one
currently repeating.

In the case of:

- key1 press
- key1 repeat
- key2 press -> key1 repeat stopped
- key2 repeat
- key2 release

The behavior should be to cancel keyboard repeat, though key1 is still
held down. This is consistent with prior X11/XWayland behavior.

The following also must work:

- key1 press
- key2 press
- key2 release
- key2 press
- key1 release
- key2 should continue to repeat

The fix for bug #778019 should continue to work:

- key1 press
- key1 repeat
- key2 press -> key1 repeat stopped
- key1 release
- key2 should repeat

The choice to change the counter nkeys to the flag repeat_active
helps to solve the second test case.

https://bugzilla.gnome.org/show_bug.cgi?id=781285

wayland: fix xdg_surface test in move/resize drag

begin_resize_drag() and begin_move_drag() check for xdg_surface being
not null, but those apply on xdg_toplevel so they should check for
xdg_toplevel being non-null instead.

https://bugzilla.gnome.org/show_bug.cgi?id=781945

wayland: check parent surface for subsurface

When an event is received while a tooltip is showing, the GtkTooltip's
event handling code can end up calling gdk_window_set_transient_for()
from gtk_tooltip_set_last_window().

The Wayland GDK backend will try to automatically create a subsurface
in gdk_wayland_window_set_transient_for() but if the parent surface is
gone meanwhile, this will will cause a crash when trying to create a
subsurface from a parent with a null surface.

Checking for the parent is not sufficient, we ought to check for the
parent surface as well to avoid the crash.

https://bugzilla.gnome.org/show_bug.cgi?id=782283

wayland: Do not map toplevel utility as popup

Applications can specify the type hint as utility even on toplevel
windows.

When that toplevel is also marked as a transient for another window,
GDK Wayland backend would translate that as an xdg_popup which is not
appropriate.

While utility temp windows should remain mapped as subsurfaces (such as
the ones used by treeviews), regular windows should not translate as
neither a subsurface nor an xdg_popup.

https://bugzilla.gnome.org/show_bug.cgi?id=781945

gtkfilechoosernativewin32: Fix support for non-ASCII paths

The code used SIGDN_URL to get an URL for the selected item, but Windows URLs
are a mix of unicode and percent encoded characters in the locale encoding
and not something GFile can understand. The result is a garbage file
path.

Instead use SIGDN_FILESYSPATH to get a real file path if available.

Also checks the return value of g_utf16_to_utf8 because file paths on
Windows can contain lone surrogates which would make the conversion fail.

https://bugzilla.gnome.org/show_bug.cgi?id=783347

Redo the listbox dnd example

Rewrite this example to have better drag highlighting.

Adwaita: Add missing rule for fullscreen headerbar

Another selector forces round corners for headerbars in a stack, and it
has higher priority than the selector covering the non-stack case from
commit 712a8adbd9a11fcebce0269562e9a028a17fa862. Totem’s MainToolbar
happens to be in a stack, and we should maintain symmetry here anyway.

So, as window classes .maximized and .tiled are excluded from this other
selector, the newly handled .fullscreen case must be excluded here also.

https://bugzilla.gnome.org/show_bug.cgi?id=770513

HighContrast: Regenerate CSS files with sassc

HighContrast: Update parse-sass.sh; remove Gemfile

Make parse-sass.sh use sassc instead of the old Ruby method.

Adwaita: Remove unused Gemfile

themes: Square corners on fullscreen win headerbar

Totem uses a fullscreen window with a headerbar at the top, and without
this change, that headerbar has rounded corners, which look different
from a maximised window and let video content show through beneath.

https://bugzilla.gnome.org/show_bug.cgi?id=770513

Updated Lithuanian translation

wayland: Don't warn if the display is lost, skip exit handlers

There is no need to have every application log a warning when the
Wayland display server goes away, and we are using _exit instead of
exit elsewhere.

This is also what the X11 backend does (see gdk_x_io_error).

https://bugzilla.gnome.org/show_bug.cgi?id=745289

Update Serbian translation

wayland: Don't abort when preparing the source if connection is lost

Aborting the application makes it look like an application bug, when
it is the expected thing to do when the Wayland display server goes
way. eg., when the user logs out. The log level is also demoted to
avoid a storm of warnings in the log from all applications whenever
this happens.

This is also what the X11 backend does (see gdk_x_io_error).

https://bugzilla.gnome.org/show_bug.cgi?id=783047

textview: Make GTK_TEXT_WINDOW_PRIVATE public

The GTK_TEXT_WINDOW_PRIVATE enumeration value is really *not* private.
Internally, it's used as a simple "invalid value" marker, and
application and library developers are supposed to use it as such in
their own code.

Let's just document it, and since the GtkTextView documentation and
internals go a long way to state the fact that it should not be used as
an argument value, let's add some pre-condition checks as well.

This commit fixes GtkSourceView's use of GTK_TEXT_WINDOW_PRIVATE as
default value for a GObject property that was broken by the change in
glib-mkenums to honor the `/*< public >*/` and `/*< private >*/`
trigraphs.

tests: Add testforeign

Add a test for exporting a handle. There are no GTK+ API for this, but
only per backend GDK API, and so far only Wayland is supported. There
is a private GdkWindow API but it's not exposed externally.

https://bugzilla.gnome.org/show_bug.cgi?id=782325

GdkWaylandWindow: Allow calling export() multiple time

Allow getting the same export handle multiple times by calling
gdk_wayland_window_export_handle() multiple times. For each time
export() is called, a unexport() must be called to unexport.

When the window is already exported, the exported callback is called
via a idle handler. If there are multiple export() calls, they are
invoked in order either when the handle is received by the display
server, or when the idle callback is invoked.

Calling unexport() will not affect future invokations of the exported
callback, unless all export() calls have their unexport() call count
matched.

https://bugzilla.gnome.org/show_bug.cgi?id=782325

Move style change for popover creation.

Creating with `gtk_popover_new_from_model` should be exactly the same as
if via `gtk_popover_new` plus `gtk_popover_bind_model`.

Also remove the style if the model is unbound at any point.

widget: Remove useless assignment

We don't use adjusted_allocation after this line.

examples/drawing: Don't call gtk_main_quit

This is a GtkApplication so it will do the right thing when closing the
window.

fallback-c89: Include config.h

It holds all the HAVE_<func> definitions.

widget: remove double assignment

We never read the value we assign here.

placesview: Remove duplicate GtkMountOperation assignment

we assign a new mount operation a few lines later again.

levelbar: Don't allocate 0 blocks

Could result in a division by zero later on and doesn't make a lot of
sense anyway.

grid: Remove unnecessary NULL checks

The minimum and natural pointers passed to measure are never NULL and
that's the only place where we call gtk_grid_get_size_for_size.

GtkGestureMultiPress: Remove superfluous NULL check

The rect parameter in gtk_gesture_multi_press_set_area is annotated as
nullable and the code handles the rect==NULL case, but the
g_return_if_fail kept that case from ever happening.

label: Remove dead ternary operator

filechooserwidget: Remove always-false error case

_gtk_search_engine_new never returns NULL, it always returns a valid
GObject with at least the simple search engine initialized.

sidebarrow: Remove useless size group

Nice try, but size groups don't work with invisible widgets anyway.
Invisible widgets request 0×0.

label: Remove useless if statements

We never pass nullable pointers into get_size_for_allocation.

gtkwindow: Hide CSD title bar when undecorated

When switching from decorated to undecorated, the title bar should be
hidden as well.

https://bugzilla.gnome.org/show_bug.cgi?id=782117

docs: Fix copy-paste error in gtk_file_chooser_get_extra_widget() doc comment

Adwaita: Avoid accindental border on last treeview header

The :last-child selector supposed to reset the border was
overridden by the :hover selector. This is fixed by moving the
:last-child selector after the overriding one.
Thanks to Sebastian Keller for spotting.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=779078.

Adwaita: regenerate the css with sassc

Adwaita: modify parse-sass.sh script to use sassc

EntryBuffer: Don't generate changed events when input is truncated

widget-factory: Use :relief=none, not class .flat

The rest of the ui file follows that convention.

3.22.15

listbox: Manually unparent placeholder in dispose

We can't make the placeholder a non-internal child as that breaks
applications that previously relied on foreach() to only return
GtkListBoxRow instances. Instead, unparent the placeholder manually in
dispose.

https://bugzilla.gnome.org/show_bug.cgi?id=782494

Revert "listbox: Properly remove placeholders"

This reverts commit b7fdc5b447b44b5b7638a3ac41cac58136bc5ca2.

This was found to break the expectations of existing listbox
users.

Adwaita: regenerate the css

Adwaita: restyle entry-tag

Fix the sizing and spacing, blue tags for the bright variant,
similar to what gnome-documents was shipping, and inverted gray
tags for the dark variant, not vanishing on hover.

3.22.14

gtkclipboard: Fix typo

The standard atom is UTF8_STRING.

build: Fix the check on sassc's presence

If sassc is not available we do not want to include the rules to
regenerate the CSS out of the SCSS files.

build/win32/replace.py: Fix replacing items in files with UTF-8 content

Some files that this script will process might have UTF-8 items in
there, which can cause problems on Python 3.x as it is more strict and
careful on unicode issues.

Fix this by:
-Doing what we did before on Python 2.x
-Opening the file with encoding='utf-8' on Python 3.x

3.22.13

Fix the build

It seems that new gtk-doc doesn't like this variable,
so drop it.

GtkWindow: Don't double free export user data

The user data passed when exporting a Wayland window was supposed to be
freed using the destroy_func, as is commonly done. This was previously
broken, as the user data was just NULL:ed when exported, and only
actually destroyed when unexporting before having exported.

While e016d9a5dba6f6f99aee94d0b72c00bee299b96a fixed this, it introduced
a regression, as GtkWindow was nice enough to free the memory anyway
after having received the exported handle, causing it now to double
free.

https://bugzilla.gnome.org/show_bug.cgi?id=782109

build: Disable deprecation warnings in broadway

Avoid compiler warnings while building code that uses deprecated API.

Specify more options to sassc

We should use the compact style, to minimize the whitespace; and we
should omit the source map URL comment as well.

build: De-duplicate options to sassc

If we want to change the list of options we use with sassc, not having
to copy-paste them in multiple rules would be a good start.

build: Enable subdir-objects

Like we did in the master branch.

Using subdir-objects removes the forward compatibility warnings coming
from autotools.

build: Remove unused rules

The foreign drawing example was moved to the GTK demos.

build: Don't include the marshallers header file

We used to inject the inclusion of the generated header file into the
generated body of the marshallers source code in order to avoid compiler
warnings about missing prototypes. The glib-genmarshal utility has been
fixed in GLib to include the prototype in the generated source, so now
we're going to trip -Werror=redundant-decls.

Adwaita: regenerate css

Adwaita: add a comment

Add a comment pointing to the related bug for the dropdown menu
margin.

Revert "Adwaita: remove unneeded margin on dropdown menu"

This reverts commit bb1679b247a8805a9c87a41dcd10a2ae8b2d65d3.
The margin was actually related to:
https://bugzilla.gnome.org/show_bug.cgi?id=591258

Revert "Adwaita: regenerate the css"

This reverts commit f2f09b957441e26b0fe9ce51626b086f2b12e2c5.

Make sure to remove tooltip timeout.

Otherwise in GC-ed environments the `g_source_remove` call during
disposal might be called on an already removed source, which results in
unnecessary console output.

https://bugzilla.gnome.org/show_bug.cgi?id=778301

Adwaita: regenerate the css

Adwaita: remove unneeded margin on dropdown menu

That margin was probably a left over to workaround some placement
issue on dropdowns which doesn't exist anymore.

Update Italian translation

GdkWaylandWindow: Unexport when finalizing

The application might not have exported, and it'll be too late for it
at this point anyway, so lets be helpful and not leak it.

https://bugzilla.gnome.org/show_bug.cgi?id=782109

box: Initialize size arrays to 0

Thats the right thing to do.

https://bugzilla.gnome.org/show_bug.cgi?id=773814

style entry-tags

https://bugzilla.gnome.org/show_bug.cgi?id=781214

wayland: Remove self assignment

Don't assign the value of a variable to itself. It was added just for
clarity, but it makes coverity complain, so remove it.

https://bugzilla.gnome.org/show_bug.cgi?id=780301

wayland: Use correct enum type and values

Use the gravity enum values when converting to gravity. It doesn't fix
anything, since the enum values were identical, but it makes a coverity
warning go away.

https://bugzilla.gnome.org/show_bug.cgi?id=780301

file filter: Be save against NULL

g_content_type_from_mime_type may return NULL, deal with that.

https://bugzilla.gnome.org/show_bug.cgi?id=782180

Revert "GdkWaylandWindow: Unexport when finalizing"

This reverts commit ab4e4bd3aec6aeb3df6de16bd12d7f538de39061.

GdkWaylandWindow: Unexport when finalizing

The application might not have exported, and it'll be too late for it
at this point anyway, so lets be helpful and not leak it.

https://bugzilla.gnome.org/show_bug.cgi?id=782109

GdkWaylandWindow: Clear export user data when used

It was only cleared when unexported, but we could just as well clear it
when its used too.

https://bugzilla.gnome.org/show_bug.cgi?id=782109

linkbutton: Fix memory leak

strings returned by gtk_widget_get_tooltip_text() and
gtk_widget_get_tooltip_markup() has to be freed when no longer
required.

https://bugzilla.gnome.org/show_bug.cgi?id=782202

menubutton: Remove prelight state when button is deactivated

If the button is deactivated, it should not appear to be hovered
anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=781992

Update Hungarian translation

gtkdnd: restore drag window movement for the unmanaged case

5bb12474d975ee4b790c5 removed the dnd window movement code to let
the gdk backends handle the window movement instead. While this
works for X11/wayland the win32 backend still uses the unmanaged
interface and expects the window movement to be handled on the gtk
side. This restores the functionality in case the dnd is unmanaged.

This fixes the drag window on Windows being stuck in the top left
corner instead of following the drag position.

https://bugzilla.gnome.org/show_bug.cgi?id=781737

shortcutssection: plug a tiny mem leak

clipboard: Try the mimetype first

Try text/plain;charset=utf-8 first, before falling back to
X11-isms like UTF8_TEXT. This makes things work on Wayland
compositors that don't carry a heavy X11 legacy around.

https://bugzilla.gnome.org/show_bug.cgi?id=781814

Adwaita: regenerate css